Skip to content

pkg/machine/e2e: skip rosetta test#26293

Merged
openshift-merge-bot[bot] merged 2 commits intocontainers:mainfrom
Luap99:podman-rosetta
Jun 6, 2025
Merged

pkg/machine/e2e: skip rosetta test#26293
openshift-merge-bot[bot] merged 2 commits intocontainers:mainfrom
Luap99:podman-rosetta

Conversation

@Luap99
Copy link
Member

@Luap99 Luap99 commented Jun 5, 2025

We disabled it in the machine os image so it can no longer be used currently.

Does this PR introduce a user-facing change?

None

@openshift-ci
Copy link
Contributor

openshift-ci bot commented Jun 5, 2025

[APPROVALNOTIFIER] This PR is APPROVED

This pull-request has been approved by: Luap99

The full list of commands accepted by this bot can be found here.

The pull request process is described here

Details Needs approval from an approver in each of these files:

Approvers can indicate their approval by writing /approve in a comment
Approvers can cancel approval by writing /approve cancel in a comment

@openshift-ci openshift-ci bot added the approved Indicates a PR has been approved by an approver from all required OWNERS files. label Jun 5, 2025
@Luap99
Copy link
Member Author

Luap99 commented Jun 5, 2025

@baude @mheon PTAL, the machine os was merged and I missed this test depends on rosetta so machine CI is broken until this gets merged.

@mheon
Copy link
Member

mheon commented Jun 5, 2025

LGTM

@packit-as-a-service
Copy link

packit-as-a-service bot commented Jun 5, 2025

[NON-BLOCKING] Packit jobs failed. @containers/packit-build please check. Everyone else, feel free to ignore.

@Luap99
Copy link
Member Author

Luap99 commented Jun 5, 2025

ssh to running machine and check proxy settings

Error: machine did not transition into running state: ssh error: ssh: unexpected packet in response to channel open: <nil>

In the proxy test, that one is new I see it on #26292 as well I though it was start a flake but given it fails in this exact tests I wonder if something "bad" got into the image now that we are using the latest coreos all of the sudden...

@Luap99
pkg/machine/e2e: skip rosetta test
0c406f2 
We disabled it in the machine os image so it can no longer be used
currently.

Signed-off-by: Paul Holzinger <pholzing@redhat.com>
@Luap99
Copy link
Member Author

Luap99 commented Jun 6, 2025

Ok I found a workaround, something is very broken in the new image that setting a basic env var via environment.d causes boot failures.

@Luap99
podman machine: fix proxy test
0b8dd90 
Do not write /etc/environment.d files, something is broken in the new
image that causes the boot to fail when any basic var is set there.

Signed-off-by: Paul Holzinger <pholzing@redhat.com>
@baude
Copy link
Member

baude commented Jun 6, 2025

LGTM

@mheon
Copy link
Member

mheon commented Jun 6, 2025

/lgtm

@openshift-ci openshift-ci bot added the lgtm Indicates that a PR is ready to be merged. label Jun 6, 2025
@openshift-merge-bot openshift-merge-bot bot merged commit e8e5dce into containers:main Jun 6, 2025
82 of 83 checks passed
@Luap99 Luap99 deleted the podman-rosetta branch June 6, 2025 16:35
@stale-locking-app stale-locking-app bot added the locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. label Sep 5, 2025
@stale-locking-app stale-locking-app bot locked as resolved and limited conversation to collaborators Sep 5, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Reviewers

No reviews

Assignees

@mheon mheon

Labels

approved Indicates a PR has been approved by an approver from all required OWNERS files. lgtm Indicates that a PR is ready to be merged. locked - please file new issue/PR Assist humans wanting to comment on an old issue or PR with locked comments. machine release-note-none

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@Luap99 @mheon @baude